Skip to content

Conversation

@RaidenE1
Copy link
Contributor

@RaidenE1 RaidenE1 commented Sep 5, 2025

The original test timeout when using new protocol, because it use
ConsumerConfig.MAX_POLL_INTERVAL_MS_CONFIG as the exception's timeout,
which is 300s. Also the test for new protocol and old protocol use the
same group ID, so the failure will be hidden.
What I do:

  1. Set the timeout as 5 secs so it can be captured within 10s
  2. Use new appId for new protocol

Reviewers: Lucas Brutschy [email protected]

@github-actions github-actions bot added triage PRs from the community streams tests Test fixes (including flaky tests) small Small PRs labels Sep 5, 2025
Copy link
Member

@lucasbru lucasbru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@github-actions github-actions bot removed the triage PRs from the community label Sep 6, 2025
@lucasbru lucasbru merged commit b3eed51 into apache:trunk Sep 8, 2025
34 of 36 checks passed
@RaidenE1 RaidenE1 deleted the bug-fix branch September 18, 2025 13:39
@lucasbru lucasbru added the KIP-1071 PRs related to KIP-1071 label Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

KIP-1071 PRs related to KIP-1071 small Small PRs streams tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants